-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add example app, gen d.ts files #3
Conversation
AkifumiSato
commented
Aug 26, 2023
•
edited
Loading
edited
- appsの作成
- CIの一部名称変更
- example-nextの実装
- https://github.com/AkifumiSato/location-state-poc から移行
- d.tsの生成
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
example-nextはなぜnpm? これもpnpmでいいのでは?
【追記】じゃなくてpackage-lock.json
が残ってるだけ?
create-next-appした時に--use-pnpmつけ忘れた気がします。 |
ローカルでは通るんでCI落ちてるのはturboのキャッシュ周りっぽいですが... |
8674089
to
10541b0
Compare
コメントのissueが解決されないとd.tsの生成待たずにturborepoが次の生成に行ってしまうためにd.tsの生成は諦めて戻しました。 |
tsupまだ時期尚早ですかねー? |
rollupと比べると冗長な設定もなくプラグインいっぱい入れずに済むから導入ハードルも低くって、個人的には↑くらいならまだ全然許容な範囲の感覚です! |
ちょっとやってみたんですが、
などあり、結構作業大きくなりそうだったので一旦諦めました。 |
|
ac6dd0b
to
99f301e
Compare
たまにないパッケージもあるからなしも一瞬考えたんですがやっぱ必須ですよね、、、 |
@@ -1,14 +1,24 @@ | |||
{ | |||
"$schema": "https://turbo.build/schema.json", | |||
"pipeline": { | |||
"dts": { | |||
"dependsOn": ["^dts"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
turborepoのdependsOn
の説明勘違いしてて^
付けてなかったんですが、これが依存関係に依存する処理をturborepoに伝えるものだったので追加しました。
https://turbo.build/repo/docs/reference/configuration#dependson
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
乙でした!